components/zoho_sheet/common/utils.mjs (1)

1-2: Consider adding input type validation

While the function handles falsy values appropriately, it might be helpful to add explicit type checking for the input parameter to prevent unexpected behavior with non-serializable objects.

 export const parseObject = (obj) => {
+  if (obj === null || obj === undefined) return undefined;
+  if (typeof obj !== 'string' && !Array.isArray(obj) && typeof obj !== 'object') {
+    throw new TypeError('Input must be a string, array, or object');
+  }
-  if (!obj) return undefined;

components/zoho_sheet/sources/new-workbook-instant/new-workbook-instant.mjs (2)

4-11: Enhance the component description

The description could be more detailed to help users understand the component's capabilities better.

-  description: "Emit new event whenever a new workbook is created.",
+  description: "Emit new event whenever a new workbook is created in Zoho Sheet. Optionally monitor a specific location.",

---

17-19: Document the expected event structure

The getSummary method uses workbook_name and resource_id properties from the event object. These properties should be documented to help with maintenance and debugging.

Add JSDoc comments to describe the expected event structure:

+    /**
+     * @param {Object} event - The webhook event object
+     * @param {string} event.workbook_name - The name of the created workbook
+     * @param {string} event.resource_id - The unique identifier of the workbook
+     * @returns {string} A human-readable summary of the event
+     */
     getSummary(event) {
       return  `New workbook: ${event.workbook_name} (${event.resource_id})`;
     },

components/zoho_sheet/sources/new-row-instant/new-row-instant.mjs (1)

12-30: Consider adding required field validation

While the props are well-structured with proper dependencies, consider adding validation to ensure both workbookId and worksheetId are required fields.

 workbookId: {
   propDefinition: [
     common.props.zohoSheet,
     "workbookId",
   ],
+  required: true,
 },
 worksheetId: {
   propDefinition: [
     common.props.zohoSheet,
     "worksheet",
     ({ workbookId }) => ({
       workbookId,
     }),
   ],
   withLabel: true,
+  required: true,
 },

components/zoho_sheet/sources/common/base.mjs (2)

4-16: Add input validation for serviceName prop.

While the code removes spaces during webhook creation, it would be better to validate the input earlier and provide clear feedback to users about allowed characters.

 serviceName: {
   type: "string",
   label: "Service Name",
-  description: "The name of the webhook.",
+  description: "The name of the webhook. Only alphanumeric characters are allowed.",
+  pattern: "^[a-zA-Z0-9]+$",
+  errorMessage: "Service name must contain only letters and numbers",
 },

---

17-21: Add JSDoc documentation for the getExtraData method.

Since this method serves as an extension point for derived classes, it should be properly documented.

 methods: {
+  /**
+   * Override this method to provide additional data for webhook creation/deletion
+   * @returns {Object} Additional data to be included in webhook requests
+   */
   getExtraData() {
     return {};
   },
 },

components/zoho_sheet/sources/new-or-updated-row-instant/new-or-updated-row-instant.mjs (2)

6-11: Enhance the component description.

While the current description is clear, it would be more helpful to mention that this requires worksheet configuration and explain what constitutes a "modification" (e.g., which fields trigger updates).

-  description: "Emit new event whenever a row is added or modified.",
+  description: "Emit new event whenever a row is added or modified in a specified worksheet. Modifications include changes to any cell in the row.",

---

30-34: Clarify the alert message about row completion.

The current alert message doesn't specify what constitutes a "completed" row. This could be confusing for users.

-      content: "**New row** will be triggered only after the entire row is completed.",
+      content: "**New row** events will be triggered only after all required cells in the row have values and the row is saved.",

components/zoho_sheet/actions/create-row/create-row.mjs (2)

27-32: Add validation for headerRow

Consider adding minimum value validation for the headerRow prop to prevent invalid row numbers.

 headerRow: {
   type: "integer",
   label: "Header Row",
   description: "Default value is 1. This can be mentioned if the table header is not in the first row of the worksheet.",
   optional: true,
+  min: 1,
 },

---

4-54: Consider implementing rate limiting

As this component is part of a larger Zoho Sheet integration, consider implementing rate limiting to handle API quotas and prevent throttling.

This could be implemented at the app level (zoho_sheet.app.mjs) using a rate limiter utility to ensure all components respect API limits.

components/zoho_sheet/actions/update-row/update-row.mjs (3)

4-9: Consider starting with version 1.0.0

Since this is a new component that will be published for production use, consider starting with version 1.0.0 instead of 0.0.1 to indicate a stable, production-ready release.

-  version: "0.0.1",
+  version: "1.0.0",

---

34-41: Improve criteria prop description

The description references "as described above" but there's no prior description in the component. Consider providing a clear example of the criteria format.

-      description: "If criteria is not set all available rows will get updated. Mention the criteria as described above.",
+      description: "Specify the criteria to identify rows for updating. Example: {\"columnName\":\"value\"}. If not set, all available rows will be updated.",

---

55-62: Enhance data prop description and validation

The data property's description could be improved with better formatting and validation guidance.

-      description: "The JSON data that needs to be updated. Example:{\"Month\":\"May\",\"Amount\":50}",
+      description: "The data to update in the matched rows. Provide a JSON object where keys are column names and values are the new values to set.\n\nExample:\n```json\n{\n  \"Month\": \"May\",\n  \"Amount\": 50\n}\n```\n\nNote: Values must match the column data types in your worksheet.",

components/zoho_sheet/actions/search-delete-row/search-delete-row.mjs (2)

42-47: Fix typo in rowArray description.

The description contains a spelling error: "indexs" should be "indices".

-      description: "Array of row indexs, which needs to be deleted.",
+      description: "Array of row indices, which needs to be deleted.",

---

67-87: Consider enhancing error handling and response details.

While the implementation is generally good, consider these improvements:

1. The summary message could include more details (e.g., number of rows deleted)
2. Add response validation before returning

   async run({ $ }) {
     if (!this.criteria && !this.rowArray) {
       throw new ConfigurationError("You must provide at least **Criteria** or **Row Array** to process this request.");
     }
     const response = await this.zohoSheet.deleteRow({
       $,
       workbookId: this.workbookId,
       data: {
         worksheet_id: this.worksheet,
         header_row: this.headerRow,
         criteria: this.criteria,
         row_array: JSON.stringify(parseObject(this.rowArray)),
         first_match_only: this.firstMatchOnly,
         is_case_sensitive: this.isCaseSensitive,
         delete_rows: this.deleteRows,
       },
     });
 
+    if (!response?.data) {
+      throw new Error("Unexpected API response format");
+    }
+
+    const deletedCount = response.data.deleted_count || 0;
     $.export("$summary", 
-      `Row matching criteria deleted successfully from worksheet ${this.worksheet}`
+      `Successfully deleted ${deletedCount} row(s) from worksheet ${this.worksheet}`
     );
     return response;
   },

components/zoho_sheet/zoho_sheet.app.mjs (2)

45-45: Typographical Error in Description

The description for the headerRow property ends with double periods. Please remove the extra period.

Apply this diff to fix the typo:

-          description: "By default, first row of the worksheet is considered as header row. This can be used if tabular data starts from any row other than the first row..",
+          description: "By default, first row of the worksheet is considered as header row. This can be used if tabular data starts from any row other than the first row.",

---

76-89: Clarify Parameter Naming in _makeRequest Method

In the _makeRequest method, the method parameter is used both as a function argument and within the data payload sent to the API. This might cause confusion. Consider renaming the data payload's method parameter to apiMethod or similar to distinguish it from the HTTP method.

Apply this diff to clarify parameter naming:

       _makeRequest({
         $ = this, path, data, method, ...opts
       }) {
         return axios($, {
           method: "POST",
           url: this._baseUrl() + path,
           headers: this._headers(),
           data: {
             ...data,
-            method,
+            apiMethod: method,
           },
           ...opts,
         });
       },

Also, update the calls to _makeRequest accordingly:

-        method: "workbook.list",
+        apiMethod: "workbook.list",

Repeat this change for all instances where _makeRequest is called.

Configuration used: CodeRabbit UI
Review profile: CHILL

#!/bin/bash
# Search for any Zoho Sheet API documentation or similar webhook implementations
rg -g '!node_modules' -g '!dist' -A 5 'zoho.*webhook.*payload' .
rg -g '!node_modules' -g '!dist' -A 5 'zoho.*sheet.*webhook' .

# Look for other test events in the codebase for consistency
fd -e mjs -e js 'test-event' components/zoho_sheet

#!/bin/bash
# Check contents of other test events
cat components/zoho_sheet/sources/new-or-updated-row-instant/test-event.mjs
cat components/zoho_sheet/sources/new-row-instant/test-event.mjs

# Look for webhook handling code
rg -g '!node_modules' -g '!dist' -A 10 'handleWebhook' components/zoho_sheet/

#!/bin/bash
# Look for webhook implementation
rg -g '!node_modules' -g '!dist' -A 10 'app.post.*webhook' components/zoho_sheet/
rg -g '!node_modules' -g '!dist' -A 10 'createHook|createWebhook' components/zoho_sheet/

# Check source files for documentation
cat components/zoho_sheet/sources/new-workbook-instant/new-workbook-instant.mjs

#!/bin/bash
# Check Zoho Sheet app implementation
cat components/zoho_sheet/zoho_sheet.app.mjs

# Look for any documentation files
fd -e md . components/zoho_sheet/

#!/bin/bash
# Check README content
cat components/zoho_sheet/README.md

# Look for any webhook-related test files
fd -e test.js -e test.mjs . components/zoho_sheet/

#!/bin/bash
# Description: Check for potential breaking changes in the codebase

# Check for any existing exports that might be affected
rg -l "from.*zoho_sheet" 

# Check for existing import patterns that might break
rg -l "dist/app/zoho_sheet.app.mjs"

#!/bin/bash
# Check the content of files importing zoho_sheet to understand the import patterns
for file in $(rg -l "from.*zoho_sheet"); do
  echo "=== $file ==="
  rg "from.*zoho_sheet" "$file" -A 2
done

# Check if there are any other package.json files referencing this component
rg -l "zoho_sheet" "**/package.json"

#!/bin/bash
# Description: Check for platform dependency usage and compatibility

# Check for platform imports in new components
rg "from.*@pipedream/platform"

# Check other components' platform versions for consistency
fd package.json -x jq -r 'select(.dependencies."@pipedream/platform") | .dependencies."@pipedream/platform"'

#!/bin/bash
# Description: Verify build configuration and package structure

# Check for build config files
fd -t f "rollup.config|webpack.config|tsconfig"

# Check for related build scripts in root package.json
cat package.json | jq '.scripts | select(. | test("build|dist"))'

#!/bin/bash
# Check for build config files in component directory
fd -t f "rollup.config|webpack.config|tsconfig|package.json" components/zoho_sheet/

# Check for build scripts in component's package.json
if [ -f "components/zoho_sheet/package.json" ]; then
  cat components/zoho_sheet/package.json | jq '.scripts'
fi

# Check if the file exists at the specified path
ls -l components/zoho_sheet/zoho_sheet.app.mjs

#!/bin/bash
# Description: Find all files that import and use parseObject

# Find all files that import from utils.mjs
echo "Files importing parseObject:"
rg -l "import.*parseObject.*from.*utils" components/zoho_sheet/

# Show the context of parseObject usage
echo "\nUsage context:"
rg -B 2 -A 2 "parseObject\(" components/zoho_sheet/

#!/bin/bash
# Description: Check the test event structure
# Expected: Test event should contain workbook_name and resource_id

# Display the test event structure
cat components/zoho_sheet/sources/new-workbook-instant/test-event.mjs

# Look for required properties in the test event
ast-grep --pattern 'export default {
  $$$
  workbook_name: $_,
  resource_id: $_,
  $$$
}'

#!/bin/bash
# Description: Check the common base implementation for required webhook methods
# Expected: Methods for webhook handling should be present in the base module

# Check the base module implementation
cat components/zoho_sheet/sources/common/base.mjs

# Look for webhook-related method implementations
ast-grep --pattern 'export default {
  $$$
  methods: {
    $$$
    setupWebhook($$$) {
      $$$
    }
    $$$
  }
  $$$
}'

#!/bin/bash
# Let's try a different approach to check the base implementation

# First, check if the base.mjs file exists and view its contents
cat components/zoho_sheet/sources/common/base.mjs

# Also search for any webhook-related patterns in the codebase
rg -l "setupWebhook|processEvent" --type js components/zoho_sheet/sources/common/

# Search for webhook method implementations with ripgrep for better context
rg "setupWebhook.*\{" -A 5 --type js components/zoho_sheet/sources/common/

#!/bin/bash
# Check if test event file exists and contains proper test data
cat components/zoho_sheet/sources/new-row-instant/test-event.mjs

#!/bin/bash
# Check the component implementation
cat components/zoho_sheet/sources/new-row-instant/new-row-instant.mjs

# Check the base implementation
cat components/zoho_sheet/sources/common/base.mjs

#!/bin/bash
# Description: Verify that all required webhook sources are implemented and extend this base class

# Check for the existence of webhook source files
echo "Checking webhook source implementations..."
fd -e mjs -p "new-row-instant|new-or-updated-row-instant|new-workbook-instant" components/zoho_sheet/sources/

# Verify they extend the base class
for file in $(fd -e mjs -p "new-row-instant|new-or-updated-row-instant|new-workbook-instant" components/zoho_sheet/sources/); do
  echo "\nChecking $file for base class implementation..."
  rg -A 5 "import.*base" "$file"
done

#!/bin/bash
# Description: Verify the Zoho Sheet API documentation link
curl -I "https://www.zoho.com/sheet/help/api/v2/" | grep "HTTP/"

#!/bin/bash
# Description: Check for consistency across Zoho Sheet components

# Check for consistent prop definitions across components
echo "Checking prop definitions consistency..."
rg -A 5 "propDefinition.*zohoSheet" components/zoho_sheet/

# Check for consistent error handling patterns
echo "Checking error handling patterns..."
rg "throw new Error" components/zoho_sheet/

# Check for consistent version numbers
echo "Checking version numbers..."
rg '"version":' components/zoho_sheet/

#!/bin/bash
# Check for consistent usage of common utilities and prop definitions
echo "Checking common utilities usage..."
rg -l "parseObject" components/zoho_sheet/

echo "\nChecking workbook and worksheet prop definitions..."
ast-grep --pattern 'propDefinition: [
  zohoSheet,
  "workbookId"
]'

echo "\nChecking data handling in actions..."
rg -A 3 "JSON.stringify" components/zoho_sheet/

echo "\nChecking component versions..."
fd -e mjs$ . components/zoho_sheet/ -x grep -l "version"

#!/bin/bash
# Description: Verify the Zoho Sheet API documentation link
curl -I "https://www.zoho.com/sheet/help/api/v2/" | grep "HTTP/"

#!/bin/bash
# Description: Verify the correct format for the Authorization header in Zoho Sheet API.

# Test: Fetch the API documentation for Zoho Sheet authentication headers.
curl -s 'https://www.zoho.com/sheet/api/' | grep -i 'Authorization'

#!/bin/bash
# Search for other instances of Zoho API authorization headers in the codebase
rg -i "Zoho-oauthtoken" -B 2 -A 2

# Search for any Zoho API related documentation or configuration files
fd -e md -e txt -e json | xargs rg -i "zoho.*auth"

# Look for similar authorization patterns in other Zoho-related files
fd zoho | xargs rg -i "authorization.*header"